-
Notifications
You must be signed in to change notification settings - Fork 38.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move LoadPodFromFile to volume utils #47514
Move LoadPodFromFile to volume utils #47514
Conversation
e57126c
to
7bd2c91
Compare
@smarterclayton, you're author of the original issue, PTAL (and approve) |
I wonder how removal of pkg/util/io/io.go caused these golint errors to appear. Fixed in the second patch. |
/assign @smarterclayton |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jsafrane, smarterclayton Associated issue: 16970 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
e1b9054
to
f3ad626
Compare
rebased |
SavePodToFile is not used anywhere and LoadPodFromFile is used only by PV recycler.
rebased (restoring lgtm...) |
Removal of io.go revealed new lint errors in pkg/util/io
f3ad626
to
c2dc5b5
Compare
87ffa1c
to
b73517d
Compare
Automatic merge from submit-queue (batch tested with PRs 48074, 47971, 48044, 47514, 47647) |
SavePodToFile is not used anywhere and LoadPodFromFile is used only by PV
recycler.
Fix #16970